src/Projects/BKPresentation/Sources/MainFlow/Note/View/EmotionRegistrationView.swift (1)

26-27: default 대신 명시적 케이스 매칭 고려

새로운 .etc 감정 케이스 추가로 인해 default가 필요해진 것으로 보입니다. 하지만 default를 사용하면 향후 Emotion에 새로운 케이스가 추가될 때 컴파일러 경고 없이 자동으로 warmEmotion으로 매핑되어 버그를 놓칠 수 있습니다.

default 대신 case .etc:(또는 현재 해당하는 케이스)를 명시적으로 매칭하면, 새 케이스 추가 시 컴파일 에러로 누락을 방지할 수 있습니다.

제안

-        default:
+        case .etc:
             imageView.image = BKImage.Graphics.warmEmotion

두 switch 문 모두 동일하게 적용하면 됩니다.

Also applies to: 159-160

src/Projects/BKCore/Sources/Extension/Optional.swift (1)

3-3: 불필요한 UIKit import

이 파일은 UIKit 타입을 사용하지 않습니다. Foundation으로 충분합니다.

♻️ 수정 제안

-import UIKit
+import Foundation

src/Projects/BKPresentation/Sources/MainFlow/NoteCompletion/View/AppreciationResultView.swift (1)

33-34: default 대신 .etc를 명시적으로 매칭하는 것을 고려해 주세요.

현재 default는 향후 Emotion에 새로운 케이스가 추가될 때 컴파일러 경고 없이 .warmth로 매핑됩니다. .etc를 명시적으로 처리하면 새 케이스 추가 시 컴파일 타임에 누락을 감지할 수 있습니다.

♻️ 수정 제안

         case .insight:
             return .insight
-        default:
+        case .etc:
             return .warmth

src/Projects/BKDomain/Sources/Entity/RecordFetchResult.swift (1)

3-3: Foundation import가 불필요해 보입니다.

이 struct는 Foundation 타입을 직접 사용하지 않습니다. RecordInfo와 Emotion은 같은 모듈 내 타입이므로 import 없이도 접근 가능합니다.

♻️ 제안

-import Foundation

src/Projects/BKPresentation/Sources/MainFlow/BookDetail/View/SeedItemView.swift (1)

57-62: emotion이 nil일 때 dot이 투명하게 표시됩니다.

EmotionSeed.from(seed:)가 nil을 반환하면 dotView.backgroundColor가 nil(투명)이 됩니다. SeedGraphView에서는 같은 경우에 .lightGray 폴백을 사용하고 있어 동작이 일관되지 않습니다. 동일한 폴백 색상 적용을 고려해 주세요.

♻️ 폴백 색상 추가 제안

-        dotView.backgroundColor = emotion?.graphTintColor
+        dotView.backgroundColor = emotion?.graphTintColor ?? .lightGray

src/Projects/BKPresentation/Sources/MainFlow/BookDetail/View/SeedGraphView.swift (1)

16-16: cornerRadius 매직 넘버.

cornerRadius = 6이 하드코딩되어 있습니다. SeedItemView처럼 BKRadius 디자인 토큰 사용을 권장합니다.

src/Projects/BKDesign/Sources/Foundation/ColorSystem/BKSemanticColor.swift (1)

142-142: Emotion.etc의 resolve 패턴이 다른 케이스들과 일관성이 없습니다.

다른 케이스들은 BKAtomicColor.X.y.color처럼 정적 UIColor를 직접 반환하지만, .etc는 bkContentColor(.secondary)를 사용하여 traitCollection 기반의 동적 UIColor를 반환합니다. 기능적으로는 동작하지만, resolve(for:) 패턴에서 일관성이 깨집니다. 예를 들어, 다른 곳에서 resolve(for: .light)의 결과가 정적 색상일 것이라 가정하고 비교하는 경우 예상치 못한 동작이 발생할 수 있습니다.

♻️ 직접 atomic color를 반환하도록 변경 제안

-                case .etc: return .bkContentColor(.secondary)
+                case .etc: return BKAtomicColor.Neutral.n500.color

src/Projects/BKPresentation/Sources/MainFlow/BookDetail/Models/EmotionSeed.swift (1)

21-21: .etc 케이스의 이미지 리소스가 다른 감정의 것을 재사용하고 있습니다.

image는 sad의 이미지, circleImage는 insight의 원형 이미지, cardImage는 sad의 카드 이미지를 사용하고 있습니다. 의도적인 임시 처리라면 괜찮지만, "기타" 감정에 슬픔이나 깨달음 관련 이미지가 보이면 사용자에게 혼동을 줄 수 있습니다. "기타"용 전용 이미지 에셋이 필요한지 디자인팀과 확인해 보시는 것을 권장합니다.

Also applies to: 31-31, 97-97

src/Projects/BKPresentation/Sources/MainFlow/BookDetail/View/SeedReportView.swift (3)

124-129: labelsStackView 높이에 매직 넘버 72가 사용되고 있습니다.

다른 레이아웃 값들은 LayoutConstants에 정의되어 있지만, 이 값만 인라인으로 하드코딩되어 있습니다. 일관성을 위해 상수로 추출하는 것을 권장합니다.

♻️ LayoutConstants에 추가

 enum LayoutConstants {
     // ...
+    static let labelsHeight: CGFloat = 72
     static let graphHeight: CGFloat = 12
     // ...
 }

 labelsStackView.snp.makeConstraints {
     $0.top.equalTo(graphView.snp.bottom).offset(LayoutConstants.labelsTopOffset)
-    $0.height.equalTo(72)
+    $0.height.equalTo(LayoutConstants.labelsHeight)
     $0.horizontalEdges.equalToSuperview()
     $0.bottom.equalToSuperview()
 }

---

132-139: emotion 파라미터명이 로컬 변수에 의해 가려지고 있습니다(variable shadowing).

Line 132의 파라미터 emotion: Emotion이 line 133의 let emotion = EmotionSeed.from(emotion: emotion)에 의해 가려집니다. 동작에는 문제 없지만 가독성을 위해 로컬 변수명을 분리하는 것이 좋습니다.

♻️ 변수명 분리 제안

 func setEmotionHeader(with emotion: Emotion) {
-    let emotion = EmotionSeed.from(emotion: emotion)
-    let emotionText = "\'\(emotion.rawValue)\'"
-    emotionImageView.image = emotion.circleImage
-    reportLabel.highlightColor = emotion.color
+    let emotionSeed = EmotionSeed.from(emotion: emotion)
+    let emotionText = "\'\(emotionSeed.rawValue)\'"
+    emotionImageView.image = emotionSeed.circleImage
+    reportLabel.highlightColor = emotionSeed.color
     reportLabel.setText(text: "\(emotionText) 감정을 많이 느꼈어요")
     reportLabel.highlightedWord = emotionText
 }

---

163-177: 접기/펼치기 애니메이션 구현이 적절합니다.

UIStackView의 isHidden 애니메이션과 chevron 회전이 잘 결합되어 있습니다. 다만, 빠른 연속 탭 시 애니메이션 겹침이 발생할 수 있으므로, 필요하다면 UIViewPropertyAnimator나 애니메이션 진행 중 탭 무시 등의 보호 로직을 추가할 수 있습니다.

src/Projects/BKPresentation/Sources/MainFlow/BookDetail/View/SeedReportView.swift (2)

132-139: emotion 파라미터명을 로컬 변수로 섀도잉하고 있습니다.

Line 133에서 let emotion = EmotionSeed.from(emotion: emotion)으로 파라미터를 같은 이름으로 덮어쓰면 가독성이 떨어집니다.

♻️ 리네이밍 제안

 func setEmotionHeader(with emotion: Emotion) {
-    let emotion = EmotionSeed.from(emotion: emotion)
-    let emotionText = "\'\(emotion.rawValue)\'"
-    emotionImageView.image = emotion.circleImage
-    reportLabel.highlightColor = emotion.color
-    reportLabel.setText(text: "\(emotionText) \(emotion.descriptionText)")
+    let emotionSeed = EmotionSeed.from(emotion: emotion)
+    let emotionText = "\'\(emotionSeed.rawValue)\'"
+    emotionImageView.image = emotionSeed.circleImage
+    reportLabel.highlightColor = emotionSeed.color
+    reportLabel.setText(text: "\(emotionText) \(emotionSeed.descriptionText)")
     reportLabel.highlightedWord = emotionText
 }

---

163-176: private extension 내의 @objc private func에서 private 키워드가 중복됩니다.

private extension 내 멤버는 이미 private이므로 함수에 별도 private 지정이 불필요합니다.

♻️ 중복 접근 제어자 제거

-    `@objc` private func toggleExpansion() {
+    `@objc` func toggleExpansion() {